Skip to content

feat: added mobile reponsiveness to the upload page#936

Open
Harsh-vardhan09 wants to merge 1 commit into
magic-peach:mainfrom
Harsh-vardhan09:feat/added-mobile-responsiveness
Open

feat: added mobile reponsiveness to the upload page#936
Harsh-vardhan09 wants to merge 1 commit into
magic-peach:mainfrom
Harsh-vardhan09:feat/added-mobile-responsiveness

Conversation

@Harsh-vardhan09
Copy link
Copy Markdown
Contributor

Description

added mobile responsiveness for the mobile devices to the home page

Related Issue

Closes #822

Type of Contribution

  • Bug fix
  • New feature
  • Documentation update
  • GSSoC contribution

Participant Info

  • GitHub username:
  • Contribution level (Beginner/Intermediate/Advanced):

Screen Recording

reframe-22.mp4

Checklist

  • I have read the contribution guidelines
  • My changes follow the project structure
  • I have tested my changes in Chrome, Firefox, and Safari
  • bun run lint passes (no ESLint errors)
  • bunx tsc --noEmit passes (no TypeScript errors)
  • New interactive elements have aria-label / accessible names
  • No console.log statements left in
  • This PR is related to a valid issue
  • Screen recording attached above (required for UI/feature/design changes)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

@Harsh-vardhan09 is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @Harsh-vardhan09!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

🟠 GSSoC'26 PR detected — thanks for contributing under GirlScript Summer of Code 2026!

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 22, 2026

✅ PR Format Check Passed — @Harsh-vardhan09

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@github-actions github-actions Bot added gssoc'26 GirlScript Summer of Code 2026 level:intermediate Intermediate level - 35 pts type:bug Bug fix type:design UI/UX design type:docs Documentation type:feature New feature labels May 22, 2026
@Harsh-vardhan09
Copy link
Copy Markdown
Contributor Author

@magic-peach i have completed the pr you can check and merge if everythings is correct
feel free to contact if anythings is amiss

@magic-peach
Copy link
Copy Markdown
Owner

Hey @Harsh-vardhan09! Thanks for working on mobile responsiveness for the upload page.

This PR currently has merge conflicts with main. Please rebase:

git fetch origin
git rebase origin/main
# resolve any conflicts
git push --force-with-lease origin <your-branch>

Once conflicts are resolved and all CI checks pass, I'll review the changes and merge.

@magic-peach magic-peach removed type:bug Bug fix type:feature New feature type:docs Documentation labels May 23, 2026
@github-actions github-actions Bot added level:beginner Beginner level - 20 pts type:bug Bug fix type:docs Documentation type:feature New feature labels May 23, 2026
@Harsh-vardhan09 Harsh-vardhan09 force-pushed the feat/added-mobile-responsiveness branch from 5d0ee0f to c0e5715 Compare May 23, 2026 12:10
@github-actions github-actions Bot added the level:advanced Advanced level - 55 pts label May 23, 2026
@Harsh-vardhan09
Copy link
Copy Markdown
Contributor Author

@magic-peach fixed the merge conflict and all checks passed you can check and merge if any problem please tell me

@magic-peach magic-peach removed level:beginner Beginner level - 20 pts level:advanced Advanced level - 55 pts type:bug Bug fix type:docs Documentation labels May 24, 2026
@magic-peach
Copy link
Copy Markdown
Owner

@Harsh-vardhan09 This PR has merge conflicts with main that need to be resolved before it can be merged.

Please rebase your branch on the latest main:

git fetch upstream
git rebase upstream/main
# Resolve any conflicts
git push --force-with-lease origin feat/added-mobile-responsiveness

Also: In ThumbnailStrip.tsx, there is a const runId = lastRunIdRef.current; variable added inside the useEffect that is never used. Please remove it to keep the code clean.

Once rebased and CI passes, this will be ready to merge!

@github-actions github-actions Bot added level:beginner Beginner level - 20 pts type:bug Bug fix type:docs Documentation labels May 24, 2026
@Harsh-vardhan09 Harsh-vardhan09 force-pushed the feat/added-mobile-responsiveness branch from 142e0c1 to 9b564ad Compare May 24, 2026 11:14
@github-actions github-actions Bot added the level:advanced Advanced level - 55 pts label May 24, 2026
@Harsh-vardhan09 Harsh-vardhan09 force-pushed the feat/added-mobile-responsiveness branch from ed602cf to 37617b2 Compare May 24, 2026 11:18
@Harsh-vardhan09
Copy link
Copy Markdown
Contributor Author

@magic-peach solved now you can merge before more merge coflicts come please tell me if you need any thing else

@Harsh-vardhan09 Harsh-vardhan09 force-pushed the feat/added-mobile-responsiveness branch from 6072271 to 7fa5a4a Compare May 24, 2026 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc'26 GirlScript Summer of Code 2026 level:advanced Advanced level - 55 pts level:beginner Beginner level - 20 pts level:intermediate Intermediate level - 35 pts type:bug Bug fix type:design UI/UX design type:docs Documentation type:feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Enhance Mobile and Tablet Responsiveness

2 participants